-
Notifications
You must be signed in to change notification settings - Fork 0
Pokemoncards #13
Pokemoncards #13
Conversation
Added a grid of pokemon pulling the name and image from the PokeAPI api.
Added filter fundtionality for the grid of pokemon cards. Filter now saves with html storage and pulls names from PokeAPI to autoupdate incase of new added gens or types.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Godt du har med error checking!
Har du mulighet til å kommentere mer i koden? Slik av vi dødelige kan forstå mer av det som er gjort! :)
Noe vi må ta med oss videre:
- Akkurat nå plukker vi pokemon som var nye til hver generasjon
- Eldre pokemon kan faktisk ha vært med i den nye generasjonen, selv om de ikke er strengt talt nye
- Eksempelvis, Zubat og Golbat (som er samme pokemon) som ble introdusert i 1. generasjon, og Crobat (fortsatt samme pokemon) ble introdusert i 2. generasjon.
- Hvis vi velger "Gen II" og "Poison", så kommer bare Crobat opp. Er det noe mulighet med API'en å få opp de tidligere evolusjonene av Crobat?
- Evt. annen løsningen kan være å plukke ut de siste evolusjonene til hver pokemon når vi faktisk skal generere laget.
- Én pokemon kan ha to typings (eksempelvis, Fire + FIghting). Hadde det vært kult å vise typings til de som dukker opp også?
Added another filter to see pokemon from a specific game.
Added some comments to explain some of the code snippets that are not self explanitory
Added icons for the types for every pokemon by pulling their id and fetchign an icon from a github repo for pokemon
Har lagt inn noen kommentarer for ikke selvforklarende kode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!! Funker akkurat som jeg så for meg. Veldig bra
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vi kan fjerne Generation , tror det blir ryddigst hvis bruker velger bare ut ifra spill og type. Bildene ser utrolig bra ut! Artig med typing icon på hver Pokemon!
Removed the filter for generation, now pokemon can only be sorted by type and game.
|
Removed the filter for generations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Dette er bra.
Added a grid of pokemon with images and names that update with the filter.
Filter is now saved with html storage.
Gens and Types now pulled directly from PokeAPI to ensure every aspect of the grid and filter remains up to date.
closes #3
closes #5